Skip to content

Updates to the STM simulation framework#494

Closed
PawelPlesniak wants to merge 3 commits intoMu2e:mainfrom
PawelPlesniak:PawelPlesniak/AbsorberShift
Closed

Updates to the STM simulation framework#494
PawelPlesniak wants to merge 3 commits intoMu2e:mainfrom
PawelPlesniak:PawelPlesniak/AbsorberShift

Conversation

@PawelPlesniak
Copy link
Copy Markdown

@PawelPlesniak PawelPlesniak commented Jan 15, 2026 •
edited
Loading

This adds a set of updates to the STM simulation infrastructure. Details will be provided once complete.
Note to self - run tests on all written modules before uploading - there is a substantial rewrite included here.

@FNALbuild
Copy link
Copy Markdown
Collaborator

FNALbuild commented Jan 15, 2026

Hi @PawelPlesniak,
You have proposed changes to files in these packages:

  • Scripts
  • JobConfig

which require these tests: build.

@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.

⌛ The following tests have been triggered for a2e2f31: build (Build queue - API unavailable)

About FNALbuild. Code review on Mu2e/Offline.

@FNALbuild
Copy link
Copy Markdown
Collaborator

FNALbuild commented Jan 15, 2026

☀️ The build tests passed at a2e2f31.

Test Result Details
test with ✅ Command did not list any other PRs to include
merge ✅ Merged a2e2f31 at 192f431
build (prof) ✅ Log file. Build time: 04 min 31 sec
ceSimReco ✅ Log file.
g4test_03MT ✅ Log file.
transportOnly ✅ Log file.
POT ✅ Log file.
g4study ✅ Log file.
cosmicSimReco ✅ Log file.
cosmicOffSpill ✅ Log file.
ceSteps ✅ Log file.
ceDigi ✅ Log file.
muDauSteps ✅ Log file.
ceMix ✅ Log file.
rootOverlaps ✅ Log file.
g4surfaceCheck ✅ Log file.
FIXME, TODO ✅ TODO (0) FIXME (0) in 0 files
clang-tidy ✅ 0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at a2e2f31 after being merged into the base branch at 192f431.

For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

@oksuzian oksuzian requested a review from YongyiBWu January 28, 2026 03:00
YongyiBWu
YongyiBWu approved these changes Jan 28, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@YongyiBWu YongyiBWu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much for cleaning up the codes, Pawel!

@PawelPlesniak PawelPlesniak mentioned this pull request Mar 6, 2026
Merged
@PawelPlesniak
Copy link
Copy Markdown
Author

PawelPlesniak commented Mar 23, 2026

Closed at the request of @oksuzian who suggested reopening later.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@YongyiBWu YongyiBWu YongyiBWu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

build finished

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@PawelPlesniak @FNALbuild @YongyiBWu